Skip to content

Revert "[PyROOT] Use GetOptional family of C Python API where necessary"#19401

Closed
guitargeek wants to merge 1 commit intoroot-project:masterfrom
guitargeek:revert_gil_fix
Closed

Revert "[PyROOT] Use GetOptional family of C Python API where necessary"#19401
guitargeek wants to merge 1 commit intoroot-project:masterfrom
guitargeek:revert_gil_fix

Conversation

@guitargeek
Copy link
Contributor

@guitargeek guitargeek commented Jul 17, 2025

This reverts commit 0a823d9.

The idea is to show @wlav the test failures without that commit.

@guitargeek
Copy link
Contributor Author

Superseded by #19436

@guitargeek guitargeek closed this Jul 28, 2025
@guitargeek guitargeek deleted the revert_gil_fix branch July 28, 2025 14:55
@guitargeek guitargeek restored the revert_gil_fix branch August 7, 2025 09:25
@guitargeek guitargeek reopened this Aug 7, 2025
@guitargeek guitargeek changed the title Revert "[cppyy] Correct management of GIL in concurrency tests" Revert "[PyROOT] Use GetOptional family of C Python API where necessary" Aug 7, 2025
@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Test Results

    19 files      19 suites   3d 2h 40m 9s ⏱️
 3 247 tests  3 240 ✅ 0 💤 7 ❌
60 076 runs  60 069 ✅ 0 💤 7 ❌

For more details on these failures, see this check.

Results for commit eaeb6dc.

…sary"

This reverts commit 0a823d9.

The idea is to show @wlav the test failures without that commit.
@guitargeek
Copy link
Contributor Author

I would see the Fedora rawhide pydebug failures that I wanted to show.

@guitargeek guitargeek closed this Dec 3, 2025
@guitargeek guitargeek deleted the revert_gil_fix branch December 3, 2025 23:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant